Skip to content

Conversation

jsgf
Copy link
Contributor

@jsgf jsgf commented Jun 1, 2020

--extern proc_macro is used to add the proc_macro crate to the extern
prelude for all procmacros. In general pathless --extern only references
sysroot/standard libraries and so should be exempt from
unused-crate-dependencies warnings.

r? @petrochenkov

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 1, 2020
…arnings

`--extern proc_macro` is used to add the proc_macro crate to the extern
prelude for all procmacros. In general pathless `--extern` only references
sysroot/standard libraries and so should be exempt from
unused-crate-dependencies warnings.
@jsgf jsgf force-pushed the no-unused-pathless branch from b37db44 to 3dd6f2c Compare June 1, 2020 23:29
@petrochenkov
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 2, 2020

📌 Commit 3dd6f2c has been approved by petrochenkov

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 2, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 3, 2020
Rollup of 5 pull requests

Successful merges:

 - rust-lang#72586 (split select into submodules)
 - rust-lang#72900 (Don't count pathless --extern for unused-crate-dependencies warnings)
 - rust-lang#72924 (Stabilize `std::io::Buf{Reader, Writer}::capacity`)
 - rust-lang#72942 (clean up E0641 explanation)
 - rust-lang#72945 (Updated documentation for Control Flow Guard)

Failed merges:

r? @ghost
@bors bors merged commit 7973d93 into rust-lang:master Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants